Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#7255] Add category notes #8178

Merged
merged 7 commits into from
Apr 12, 2024
Merged

[#7255] Add category notes #8178

merged 7 commits into from
Apr 12, 2024

Conversation

gbp
Copy link
Member

@gbp gbp commented Mar 26, 2024

Relevant issue(s)

Fixes #7255

What does this do?

Add category notes

Why was this needed?

So we can create topic pages to display info requests in given categories (via tags). And render notes and other "unstyled" rich text content on the page.

Screenshots

image

@gbp gbp changed the base branch from develop to 7809-add-actiontext-to-notes March 26, 2024 15:05
gbp added 7 commits April 9, 2024 12:56
Add method to the return the uppermost root category for a given child
category.

This will be useful for finding the root in the `current_klass` helper
in `Admin::CategoriesController`. Needed in order to not required the
`model_type` parameter for all controller actions.
We don't have a show view so lets redirect to the edit form.

This is needed as we will be adding notes to categories and after a
concrete note is saved it redirects to the notable show action.
Update the `#all_notes` method to check if the current class is Taggable
before combining tagged notes.
This will be used in the admin UI when creating Notes for a given
Category.

We're not yet showing the public link as this depends on the type of
category being used.
Will be used to allow rich text non-note styled content on the public
category pages.
@gbp gbp force-pushed the 7809-add-actiontext-to-notes branch from a244cb1 to ddaff31 Compare April 9, 2024 11:56
@gbp gbp changed the base branch from 7809-add-actiontext-to-notes to develop April 9, 2024 13:38
@gbp gbp merged commit 21c9825 into develop Apr 12, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add notes to categories
1 participant